Merge branch 'master-disable-introspection' into 'master'

CI: disable introspection with mingw to avoid flaky builds

See merge request GNOME/gtk!1848

CI: use bash style variable expansion for the mingw cache key

It doesn't understand the powershell syntax which leads to the cache
key being empty which leads to cache restore being skipped
which leads to slow builds not using ccache.

See https://gitlab.com/gitlab-org/gitlab-runner/-/issues/2747

Merge branch 'wip/jimmac/HC-focus' into 'master'

HC: more contrasty focus rings

See merge request GNOME/gtk!1845

Merge branch 'abolish-size-allocate' into 'master'

Abolish size allocate

Closes #2705

See merge request GNOME/gtk!1833

Update CONTRIBUTING.md

Point to discourse, rather than mailing lists.
Based on a suggestion by sujiniku,

https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/1763

Merge branch 'matthiasc/for-master' into 'master'

keymap: Remove leftover code

See merge request GNOME/gtk!1844

CI: disable introspection with mingw to avoid flaky builds

g-i has started to fail and it's not clear why, so disable it for
now to keep the build green.

See https://gitlab.gnome.org/GNOME/gobject-introspection/-/issues/340

Merge branch 'wip/jimmac/dark-seuggested' into 'master'

Adwaita: tone down suggested focus ring for dark

See merge request GNOME/gtk!1843

widget: Drop the ::size-allocate signal

This signal is mainly used for bad things, and we
don't want to emit signals during layout if we can
avoid it.

If you are subclassing, you should either use a
layout manager or override the size_allocate vfunc.

If you are using a GtkDrawingArea or GtkGLArea,
use their ::resize signals to learn about size
changes.

Fixes: #2705

tooltipwindow: Stop using the ::size-allocate signal

Instead of connecting to ::size-allocate, call
gtk_native_set_tooltip and rely on the native
to allocate the tooltip window.

tooltipwindow: Drop some dead code

tooltip: Add private api to allocate the window

Add private gtk_tooltip_maybe_allocate() function
and use it from GtkWindow and GtkPopover.

This will let us stop using the ::size-allocate signal,
without having to redo all the tooltip management first.

That will happen later.

Explicitly update widget a11y bounds when allocating

The a11y machinery is using signal subscription to get notified of size
changes and notify listeners in turn. This is suboptimal for a couple of
reasons:

 - if something connects to the GtkWidget::size-allocate signal we need
   to emit it; currently, we have an optimization in place that will
   skip the signal emission if there are no handlers, and it would be
   nice to go through the fast path
 - the accessibility implementation is part of GTK, and should not go
   through additional hoops like any out-of-tree API consumer

tests: Stop using ::size-allocate

inspector: Stop using ::size-allocate

This signal is going away. Currently, there is no
other way for the inspector to monitor the allocation.

examples: Use GtkDrawingArea::resize

This signal is a replacement for this (ab)use
of the ::size-allocate signal, which is going away.

gtk-demo: Use GtkDrawingArea::resize

This signal is a replacement for this (ab)use
of ::size-allocate.

drawingarea: Add a ::resize signal

This can be used as a replacement for ::size-allocate
to prepare size-dependent resources, like a backing
surface. It mirrors the GtkGLArea::resize signal.

a11y: Avoid a gratitious use of ::size-allocate

The paned widget has a position property, just
use it.

cellarea: Cosmetics

Remove an occurrence of size-allocate in a comment.

treeview: Cosmetic change

Remove an occurrence of size-allocate in a comment.

window: Remove some outdated info from the docs

Remove references to ::size-allocate and ::configure-event
from the docs.

keymap: Remove leftover code

When I removed the 0-termination for the entries for
each keyval, I remove the code initializing the key,
but accidentally left the code that adds it to the
array, so gdk_keymap_get_cached_entries_for_keyval
returns one extra, uninitialized value at the end
of the array. Stop doing that.

HC: more contrasty focus rings

keymap: Remove leftover code

When I removed the 0-termination for the entries for
each keyval, I remove the code initializing the key,
but accidentally left the code that adds it to the
array, so gdk_keymap_get_cached_entries_for_keyval
returns one extra, uninitialized value at the end
of the array. Stop doing that.

Adwaita: tone down suggested focus ring for dark

Merge branch 'wip/exalm/focus-entries2' into 'master'

Adwaita: Make error/warning entry focus more subtle

See merge request GNOME/gtk!1842

Adwaita: Make error/warning entry focus more subtle

Add 50% alpha to it like to the main focus color.

Merge branch 'wip/jimmac/suggested-destructive-focus' into 'master'

Adwaita: suggested destrictive button focus

Closes #2704

See merge request GNOME/gtk!1841

Adwaita: suggested destrictive button focus

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/2704

Merge branch 'wip/jimmac/global-var' into 'master'

Adwaita: make parser happy wrt global variables

Closes #2703

See merge request GNOME/gtk!1840

Merge branch 'fix-inspector-popover' into 'master'

inspector: Fix css tree popovers

See merge request GNOME/gtk!1838

Adwaita: make parser happy wrt global variables

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/2703

Merge branch 'wip/baedert/for-master' into 'master'

Wip/baedert/for master

See merge request GNOME/gtk!1828

inspector: Fix css tree popovers

Adding popovers to a treeview doesn't work. Instead,
attach them to the cssnodetree widget, which is using
a layout manager - popovers work in that case.

Merge branch 'abolish-destroy' into 'master'

Abolish destroy

See merge request GNOME/gtk!1835

widget: Drop the destroy vfunc

We still emit the ::destroy signal, but if you
want a vfunc, use dispose.

gtk-demo: Use dispose instead of destroy

The destroy vfunc is going away.

container: Use dispose instead of destroy

The destroy vfunc is going away.

placesview: Use dispose instead of destroy

The destroy vfunc is going away.

printunixdialog: Use dispose instead of destroy

The destroy vfunc is going away.

window: Use dispose instead of destroy

The destroy vfunc is going away.

treeview: Use dispose instead of destroy

The destroy vfunc is going away.

textview: Use dispose instead of destroy

The destroy vfunc is going away.

statusbar: Use dispose instead of destroy

The destroy vfunc is going away.

notebook: Use dispose instead of destroy

The destroy vfunc is going away.

iconview: Use dispose instead of destroy

The destroy vfunc is going away.

shortcutssection: Use dispose instead of destroy

The destroy vfunc is going away.

radiobutton: Use dispose instead of destroy

The destroy vfunc is going away.

range: Use dispose instead of destroy

The destroy vfunc is going away.

spinbutton: Use dispose instead of destroy

The destroy vfunc is going away.

viewport: Use dispose instead of destroy

The destroy vfunc is going away.

text: Use dispose instead of destroy

The destroy vfunc is going away.

modelbutton: Use dispose instead of destroy

The destroy vfunc is going away.

label: Use dispose instead of destroy

The destroy vfunc is going away.

filechooserbutton: Use dispose instead of destroy

The destroy vfunc is going away.

expander: Use dispose instead of destroy

The destroy vfunc is going away.

combobox: Use dispose instead of destroy

The destroy vfunc is going away.

actionbar: Use dispose instead of destroy

The destroy vfunc is going away.

Merge branch 'misc-cleanups' into 'master'

Misc cleanups

See merge request GNOME/gtk!1832

Merge branch 'wip/exalm/controls-dispose' into 'master'

windowcontrols: Unparent children in dispose rather than finalize

See merge request GNOME/gtk!1831

widget-factory: Use symboli icons in the toolbar

That is the style that is commonly used nowadays, even
in traditional apps.

widget-factory: Remove unused css

We no longer show the various frame styles.

window: Lower the visible focus timeout

Five seconds felt too long for some people in
initial feedback, so lets lower the timeout to
3 seconds.

assistant: Drop the Private struct

placesview: Drop the Private struct

searchbar: Drop the Private struct

windowcontrols: Unparent children in dispose rather than finalize

Merge branch 'wip/jimmac/notebook-tab-focus' into 'master'

Adwaita: notebook tab focus ring

See merge request GNOME/gtk!1830

Merge branch 'bin-removal' into 'master'

Bin removal

See merge request GNOME/gtk!1820

Adwaita: notebook tab focus ring

- prelight the tab, not a label (visually)

Merge branch 'wip/jimmac/oldstyle-toolbars' into 'master'

Adwaita: old style toolbar buttons

See merge request GNOME/gtk!1829

Adwaita: old style toolbar buttons

- style toolbar buttons

shortcutcontroller: Check for NULL GtkNative of widget

And just continue in that case.

label: Be more defensive against non-existant focus_link

surface: remove double assignment

shortcutmanager: Speed up default_remove_controller a bit

Only call get_n_items on the model once and remove two unnecessary
casts.

shortcutcontroller: Use a GArray instead of a linked list

cssprovider: Pass a GtkArray around when matching

We just use the resulting GPtrArray to set the values on a GtkCssLookup
and it had mostly a size of 1, in extremes a size of 11.

gtk.supp: Update

Make some rules more general and add some more epoxy/mesa suppressions.

cssselector: Use a GtkArray to count exact matches

The list here is at most 3 elements long in my tests.

add GtkArray

Try to use stack space if we can and only fall back to the GPtrArray if
we must.

main: Save some type checks

We already have the typecheck-less versions of get_realized(),
get_parent() and is_sensitive() in gtkwidgetprivate.h, so use them here.

eventcontroller: Remove some unnecessary casts

Instead of comparing two GtkWidget*s by casting the two GtkNative*s, we
can as well compare two GtkNative*s. Also if gtk_widget_get_native()
returns NULL, the code previously failed.

widget: Use an unchecked cast in _get_native()

This is either no chagne because we know for a fact that the returned
value is a GtkNative - after all thats's the type we pass to
gtk_widget_get_ancestor().

Or it is a bug fix since casting NULL to a GtkNative using GTK_NATIVE()
is not going to work, but the API contract of gtk_widget_get_native()
explicitly allows a NULL return value.

window: Remove unused private function

scalebutton: Shorten finalize() implementation

scalebutton: Remove unused click_id member

widget: Save controllers list in a GPtrArray

widget: Only do a IS_ROOT type check if the parent is unset

So we do less type checks

placesview: Remove frame

We don't need a double border here.

cssselector: Avoid some GList allocations

icontheme: Save dir_mtimes in a GArray, not a GList

GdkEvent: Save history in a GArray

Instead of a less efficient GList.

stylecontext: Remove _list_classes() from public API

Applications can use gtk_widget_get_css_classes these days.

textchild: Return an array from get_widgets

Yay, one GList less.

gdkdisplay-wayland: Fix a possibly uninitialized out value

widget: Use early-out style in gtk_widget_realize()

This is much more common in the current code base and reduces the
indentation level by one.

shorthandproperty: Use alloca() when parsing properties

shorthandproperty: Return unset/initial/inherit as-is

We don't even need an array value these days.